Skip to content

Make the position and the size of FileSystem controls more precise#105560

Merged
akien-mga merged 1 commit intogodotengine:masterfrom
BrotherShort:Make-controls-more-precise
May 12, 2025
Merged

Make the position and the size of FileSystem controls more precise#105560
akien-mga merged 1 commit intogodotengine:masterfrom
BrotherShort:Make-controls-more-precise

Conversation

@BrotherShort
Copy link
Copy Markdown
Contributor

Before: The rename LineEdit is too long.
before1

After:
after1

Before: Text is out of hightlight backgroud.
before2

After:
after2

Before: The rename LineEdit covers the icon.
before3

After:
after3

Before: When right-to-left, The rename LineEdit and tag icons are offset.
before4

After:
after4

@BrotherShort BrotherShort requested a review from a team as a code owner April 19, 2025 09:04
@BrotherShort BrotherShort requested a review from a team April 19, 2025 09:04
@KoBeWi KoBeWi added this to the 4.5 milestone Apr 19, 2025
@KoBeWi
Copy link
Copy Markdown
Member

KoBeWi commented Apr 26, 2025

The rename LineEdit covers the icon.

Still happening:

godot.windows.editor.dev.x86_64_kngFhWX8HU.mp4

The code uses real image size for some reason.

@BrotherShort BrotherShort force-pushed the Make-controls-more-precise branch from 3cba7a5 to a386c85 Compare April 27, 2025 18:42
@BrotherShort
Copy link
Copy Markdown
Contributor Author

The rename LineEdit covers the icon.

Still happening:

godot.windows.editor.dev.x86_64_kngFhWX8HU.mp4
The code uses real image size for some reason.

Fixed.

@BrotherShort
Copy link
Copy Markdown
Contributor Author

@KoBeWi
Is there any Godot controls layout doc or diagram? Like this:
image
I drew it when i developed.

@KoBeWi
Copy link
Copy Markdown
Member

KoBeWi commented May 7, 2025

There isn't.

Looks like some conflicts were created, needs rebase.

@BrotherShort BrotherShort force-pushed the Make-controls-more-precise branch 2 times, most recently from 87bd971 to cab2993 Compare May 8, 2025 17:44
@BrotherShort
Copy link
Copy Markdown
Contributor Author

Looks like some conflicts were created, needs rebase.

Done.

@KoBeWi
Copy link
Copy Markdown
Member

KoBeWi commented May 8, 2025

Some problems with trimming caused by this PR:
image

@BrotherShort BrotherShort force-pushed the Make-controls-more-precise branch from cab2993 to a52a7a4 Compare May 10, 2025 16:32
@BrotherShort
Copy link
Copy Markdown
Contributor Author

Some problems with trimming caused by this PR: image

fixed


String _atr(int p_idx, const String &p_text) const;

protected:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Making ThemeCache protected makes it impossible for inheriting classes to implement their own. Though I see why is it needed here; I guess it's something to worry about once it actually becomes a problem (unlikely).

@akien-mga akien-mga merged commit 7d363c2 into godotengine:master May 12, 2025
20 checks passed
@akien-mga
Copy link
Copy Markdown
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants